Skip to content

Xcode 27 + Liquid Glass + SwiftCharts#2464

Open
Camji55 wants to merge 13 commits into
LoopKit:next-devfrom
Camji55:pull-request/next-dev-xcode-27
Open

Xcode 27 + Liquid Glass + SwiftCharts#2464
Camji55 wants to merge 13 commits into
LoopKit:next-devfrom
Camji55:pull-request/next-dev-xcode-27

Conversation

@Camji55

@Camji55 Camji55 commented Jul 19, 2026

Copy link
Copy Markdown
Contributor
image image image

@Camji55 Camji55 changed the title [WIP] Xcode 27 + Liquid Glass + SwiftCharts Xcode 27 + Liquid Glass + SwiftCharts Jul 19, 2026
Camji55 and others added 8 commits July 22, 2026 13:49
* Fixes presets training ctas

* Fixes presets training ctas
…e on plugin-load miss (LoopKit#2469)

* Preserve persisted pump/CGM state when a plugin can't be instantiated

instantiateDeviceManagers restored a manager by assigning pumpManager/cgmManager
= the result of ...FromRawValue(...). When the plugin wasn't available in the
running build, that returned nil, and the nil assignment fired the didSet, which
wrote rawPumpManager/rawCGMManager = nil and DELETED the persisted state — so
launching a build missing a plugin permanently wiped the pod pairing (incl.
session keys) or CGM config.

Only assign when the manager instantiates; otherwise log and leave the saved
state on disk so a later build that includes the plugin can restore it.

* Wire pump BLE heartbeat to the CGM reading schedule

Send the pump a PumpHeartbeatRequest (last CGM reading date + expected reading
interval) when it must provide the BLE heartbeat, or nil when the CGM wakes the
app itself, via the new setBLEHeartbeatRequest API. The last-reading time is
refreshed after every CGM reading (processCGMReadingResult already calls
updatePumpManagerBLEHeartbeatPreference), so the pump's heartbeat cadence tracks
the actual reading schedule.

---------

Co-authored-by: Pete Schwamb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants